Больше деталей #3
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
| * Генерация случайных комментариев для фотографии. | ||
| * Количество комментариев выбирается случайным образом (от 0 до 30). | ||
| * Каждый комментарий состоит из уникального id, сообщения, имени отправителя и аватарки отправителя. | ||
| * @returns {Array} - Массив объектов комментариев. |
There was a problem hiding this comment.
Ты обычно пишешь комменты до написания код или после?)
There was a problem hiding this comment.
В начале пишу комментаий с абстрактной информанцией о функции, например, Генерация случайного целого числа в заданном диапазоне. Затем пишу код функции. В самом конце дописываю параметры функции и проверяю соответствие комментария и функции, также дописываю какие-либо детали.
Ну обычно да, вначале пишу код, затем детальные комменты пишу.
There was a problem hiding this comment.
Просто выглядит полезно, как тз перенесенное из задания академии, чтобы не бегать сверяться 👍🏻
js/main.js
Outdated
| const comments = []; | ||
|
|
||
| for (let i = 0; i < commentsCount; i++) { | ||
| const randomMessage = messages[getRandomInt(0, messages.length - 1)]; |
There was a problem hiding this comment.
Сообщений по условию может быть и два. Но решение предлагаю придумать такое, чтобы можно было легко добавить и 3+ сообщения без значимых изменений в коде)
Да, но некоторые еще не успел посмотреть. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
js/main.js
Outdated
| comments.push({ | ||
| id: i + 1, | ||
| avatar: `img/avatar-${avatarNumber}.svg`, | ||
| message: randomMessages.join(" "), |
There was a problem hiding this comment.
Отлично, только давай по переносу строки склеим)
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
keksobot
merged commit 8911ae8
into
htmlacademy-univer-javascript-1:master
|
Pr принял, можно модуляцию делать |
🎓 Больше деталей
💥 https://htmlacademy-univer-javascript-1.github.io/2579771-kekstagram-5/3/